Using the Google library sounds reasonable to me. Rhino is under the Mozilla Public License, which is in our category B list.
- Jen On Tue, Nov 17, 2009 at 11:15 PM, Eric Dalquist <eric.dalqu...@doit.wisc.edu > wrote: > Functionally I'm fine with the approach. Do you happen to know what the > Rhino license is? > > Perhaps this is the death knell for us looking at YUI Compressor for this > work. Can we try out that new lib from Google and just have it do > compression? We can't do the optimization stuff since not all our JS is in > the files it will be pointed to. > > -Eric > > > Nicholas Blair wrote: > >> I've run into some trouble using YUI Compressor to process the >> javascript within the universality theme. >> >> Even the new version of YUI Compressor that we installed in the 3rd >> party repository (version 2.4.2) throws a >> StringIndexOutOfBoundsException in the constructor for the >> JavaScriptCompressor. >> >> I stumbled across the author's blog >> (http://www.julienlecomte.net/blog/2008/10/80/), in which he hints >> that he needed to modify Mozilla's Rhino javascript engine in order >> for it to work. >> Turns out that YUICompressor version 2.4 and later all include (!) the >> author's modified version of Rhino (see for yourself, >> org.mozilla.javascript packages bundled inside). >> >> I modified the pom.xml in maven-uportal-plugin to exclude rhino/js as >> a dependency for both yuicompressor and cernunnos, and sure enough the >> JavaScript compressor works without fail, including on the javascript >> in the universality theme. >> >> My question to the group is: is it ok to exclude the trusted rhino/js >> and depend on this author's undocumented changes? >> It's really the only way we can get YUI Compressor to work in this >> plugin. Will this present any other problems (e.g. licensing)? >> >> >> > -- Jen Bourey -- You are currently subscribed to uportal-dev@lists.ja-sig.org as: arch...@mail-archive.com To unsubscribe, change settings or access archives, see http://www.ja-sig.org/wiki/display/JSG/uportal-dev
